-
Notifications
You must be signed in to change notification settings - Fork 164
Remove duplicate start-local page #2745
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you give me some background on this change? one thing that was asked for by steve (search pm) was that you wouldn't have to change contexts too much when trying to get up and running with search just to play around. the way it was organized was pretty low maintenance as it used a single snippet.
that being said, if dev docs is ok with it I'll 🤐
deploy-manage/deploy/self-managed/local-development-installation-quickstart.md
Show resolved
Hide resolved
I'm guessing this is an SEO thing, but I share Shaina's concerns, I think having not having this page in the It's not a hill I will even start climbing though, not to mind die on 😃 |
Sure, it's tied up in efforts to clean up and get consistency in the "Solutions and use cases" > xxx > "Get started" sections. That's a thought-provoking ask from Steve so will definitely take that into consideration as we keep revising this content.
I have indeed seen the one-liner plastered around and think it's okay to continue to do so without necessarily hanging onto duplicate pages. I've re-added the curl command that existed within quickstarts to #2248 so it keeps prominence in https://www.elastic.co/docs/solutions/search/get-started |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changes in deploy-manage section look ok. will let dev docs decide whether the page removal makes sense.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No serious pushback from me, thanks for the context @lcawl
This reminds me that we probably need to add telemetry to these |
There are two copies of the
start-local
installation instructions: https://www.elastic.co/docs/solutions/search/run-elasticsearch-locally and https://www.elastic.co/docs/deploy-manage/deploy/self-managed/local-development-installation-quickstart. This PR updates links to target the latter and removes the former.This PR must be merged after elastic/elasticsearch#133691